Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CompatHelper: bump compat for Convex to 0.16 for package test, (keep existing compat) #1953

Merged

Conversation

github-actions[bot]
Copy link
Contributor

This pull request changes the compat entry for the Convex package from 0.15.4 to 0.15.4, 0.16 for package test.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@github-actions github-actions bot force-pushed the compathelper/new_version/2024-05-24-00-05-50-221-03419143078 branch from 93a7548 to b848fae Compare May 24, 2024 00:05
sloede
sloede previously approved these changes May 24, 2024
@sloede
Copy link
Member

sloede commented May 24, 2024

@ranocha do you know why no tests are running/queued?

@ranocha
Copy link
Member

ranocha commented May 24, 2024

No idea. I just merged main into this branch, so CI will tell whether it looks okay.

@DanielDoehring Any breaking changes in Convex.jl affecting your workload?

Copy link

codecov bot commented May 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.60%. Comparing base (2da0863) to head (77df414).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1953      +/-   ##
==========================================
- Coverage   96.11%   92.60%   -3.51%     
==========================================
  Files         460      460              
  Lines       36926    36894      -32     
==========================================
- Hits        35490    34164    -1326     
- Misses       1436     2730    +1294     
Flag Coverage Δ
unittests 92.60% <ø> (-3.51%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ranocha
Copy link
Member

ranocha commented May 24, 2024

@DanielDoehring CI fails due to deprecations (https://github.com/trixi-framework/Trixi.jl/actions/runs/9218674674/job/25362597322?pr=1953#step:7:825) and changes of the results (https://github.com/trixi-framework/Trixi.jl/actions/runs/9218674674/job/25362595206?pr=1953#step:7:1722). Please check whether everything is okay, update the code as required, drop support of the old version of Convex.jl, and ping me when everything is ready to be merged.

Copy link
Member

@ranocha ranocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to fix stuff as described above

@DanielDoehring
Copy link
Contributor

The differences in the Butcher array variables are marginal, so that looks ok.

Copy link
Member

@ranocha ranocha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@ranocha ranocha enabled auto-merge (squash) May 24, 2024 09:23
@ranocha ranocha merged commit 5359525 into main May 24, 2024
39 of 40 checks passed
@ranocha ranocha deleted the compathelper/new_version/2024-05-24-00-05-50-221-03419143078 branch May 24, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants